Skip to content

hw3 & hw4 - CodeProcessor, CI/CD - #2

Merged
belyaevEDU merged 78 commits into
mainfrom
hw3-hw4
Sep 7, 2026
Merged

hw3 & hw4 - CodeProcessor, CI/CD#2
belyaevEDU merged 78 commits into
mainfrom
hw3-hw4

Conversation

@belyaevEDU

@belyaevEDU belyaevEDU commented Aug 30, 2026

Copy link
Copy Markdown
Owner

че тут происходит в целом:

  • интеграция postgres для хранения данных о пользователях и тасках, redis для хранения сессий
  • rabbitmq: сервер стал producer, добавлен consumer CodeProcessor, который общается с самописанным оркестратором philharmonic. код запускается в sandbox образе, который описан в папке sandbox
  • конфигурация: распилил все конфиги, всё подгружается через .env, есть .env.example
  • весь стек (кроме philharmonic, по задумке поднимается отдельно / на отдельных машинах) поднимается docker compose
  • инструментировал сервер + CodeProcessor для prometheus-стиль метрик
  • добавил CI. go vet + golangci-lint, e2e тесты (вот те pytest) с поднятием всего стека + philharmonic, публикация sandbox образа на GHCR

fix(service/philharmonic): added back name references
@belyaevEDU
belyaevEDU marked this pull request as ready for review September 4, 2026 17:42
@KirShelestov

Copy link
Copy Markdown

ДЗ закрыто, причём заметно выше базовой планки: сборка и статанализ чистые, весь юзер флоу реально работает, тесты проходят полностью.

Ну вот что нашел, но это по мелочи:

  • в горутине metrics-сервера роняет весь процесс. Если :9100 занят, падение metrics-сервера через log.Fatalf убивает и основной HTTP-сервер. На чистом runner'е/в compose проблемы нет, но зависимость хрупкая - лучше логировать и продолжать, метрики не должны быть критичны для API.
  • Postgres везде использует context.Background() вместо проброшенного контекста

В целом, очень хорошая архитектура, разобрался с CI, все тип топ так сказать.

@belyaevEDU
belyaevEDU merged commit 63ef12b into main Sep 7, 2026
3 checks passed
@belyaevEDU
belyaevEDU deleted the hw3-hw4 branch September 7, 2026 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants